Add frdm_mcxe31b board basic support #95560

lucien-nxp · 2025-09-05T15:51:49Z

add frdm_mcxe31b board support
enable cases below:
hello_world/blinky/button/philosophers/synchronization
add mcxe31x series soc

github-actions · 2025-09-05T16:30:23Z

The following west manifest projects have changed revision in this Pull Request:

Name	Old Revision	New Revision	Diff
hal_nxp	zephyrproject-rtos/hal_nxp@`4377ecf`	zephyrproject-rtos/hal_nxp@`f33ec8f` (`master`)	zephyrproject-rtos/[email protected]

✅ All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

soc/nxp/mcx/mcxe/mcxe31x/Kconfig

nordicjm · 2025-09-18T10:04:45Z

soc/nxp/mcx/mcxe/mcxe31x/Kconfig

+config ROM_START_OFFSET
+	default 0x1000


goes in Kconfig.defconfig

nordicjm · 2025-09-18T10:05:52Z

boards/nxp/frdm_mcxe31B/CMakeLists.txt

+#
+
+if(CONFIG_NXP_MCXE31X_BOOT_HEADER)
+    zephyr_library_sources(boot_header/boot_header.c)


2 space indent for cmake

nordicjm · 2025-09-18T10:06:13Z

boards/nxp/frdm_mcxe31B/boot_header/boot_header.c

+	const uint32_t *reserved5;          /* Reserved                       */
+	const uint32_t *reserved6;          /* Reserved                       */
+	const uint32_t *lcc_config;           /* Address of LC config           */
+	uint8_t Reserved7[216];             /* Reserved for future use        */


snake case not camel case

Sorry, miss it. I have modified and use AI tool help check, there is no camel issue on boot_header files

boards/nxp/frdm_mcxe31b/frdm_mcxe31b-pinctrl.dtsi

nordicjm · 2025-09-18T10:06:43Z

boards/nxp/frdm_mcxe31b/frdm_mcxe31b.dts

+
+	leds {
+		compatible = "gpio-leds";
+		red_led: led_0 {


fix in whole PR

Check for all the dts files, it should be there is no missed newline issue for same device level definition.

I can still see a missing newline on line 37 and other places, Fix in the whole PR

hakehuang · 2025-09-22T07:37:46Z

board testing pass on v4.2.0-3891-gd777dfa367ce, with kernel and gpio testing

lucien-nxp · 2025-09-29T10:44:26Z

I will resolve conflict and sync the newest commit node on hal_nxp side after PR in hal_nxp side merged.

nordicjm · 2025-09-30T08:08:42Z

drivers/clock_control/clock_control_nxp_mc_cgm.c

+
+#if DT_NODE_HAS_COMPAT_STATUS(DT_NODELABEL(fxosc), nxp_fxosc, okay)
+const fxosc_config_t fxosc_config = {.freqHz = NXP_FXOSC_FREQ,
+							 .workMode = NXP_FXOSC_WORKMODE,


have these files been put through clang-format? Because the struct here looks very odd

nordicjm · 2025-09-30T08:09:41Z

drivers/pinctrl/Kconfig.nxp_siul2

 	default y
-	depends on DT_HAS_NXP_S32ZE_SIUL2_PINCTRL_ENABLED || DT_HAS_NXP_S32K3_SIUL2_PINCTRL_ENABLED
+	depends on DT_HAS_NXP_S32ZE_SIUL2_PINCTRL_ENABLED || DT_HAS_NXP_S32K3_SIUL2_PINCTRL_ENABLED || \
+				DT_HAS_NXP_MCXE31X_SIUL2_PINCTRL_ENABLED


Suggested change

DT_HAS_NXP_MCXE31X_SIUL2_PINCTRL_ENABLED

DT_HAS_NXP_MCXE31X_SIUL2_PINCTRL_ENABLED

nordicjm · 2025-09-30T08:10:43Z

dts/arm/nxp/nxp_mcxe31x_common.dtsi

+	cpus {
+		#address-cells = <0x1>;
+		#size-cells = <0>;
+		core0: cpu@0 {


missing newline on line 15

nordicjm · 2025-09-30T08:10:51Z

dts/arm/nxp/nxp_mcxe31x_common.dtsi

+&peripheral {
+	#address-cells = <1>;
+	#size-cells = <1>;
+	adc_0: adc@a0000 {


etc. fix in whole PR

nordicjm · 2025-09-30T08:11:19Z

boards/nxp/frdm_mcxe31b/CMakeLists.txt

@@ -0,0 +1,10 @@
+#
+# Copyright 2025 NXP


remove excess spacing from commit title

nordicjm · 2025-09-30T08:11:30Z

boards/nxp/frdm_mcxe31b/Kconfig

+# Copyright 2025 NXP
+# SPDX-License-Identifier: Apache-2.0
+
+config NXP_MCXE31X_BOOT_HEADER


BOARD_ prefix missing

nordicjm · 2025-09-30T08:11:43Z

boards/nxp/frdm_mcxe31b/boot_header/boot_header.c

+/*
+ * Copyright 2025 NXP
+ *
+ * SPDXLicense-Identifier: Apache-2.0


needs fixing

nordicjm · 2025-09-30T08:12:17Z

boards/nxp/frdm_mcxe31b/frdm_mcxe31b.dts

+
+	leds {
+		compatible = "gpio-leds";
+		red_led: led_0 {


I can still see a missing newline on line 37 and other places, Fix in the whole PR

Updated hal_nxp to add mcxe31x pinctrl.h Signed-off-by: Lucien Zhao <[email protected]>

- add clock_init function to initialize clock sources according devicetree settings - finish basic clock api function Signed-off-by: Lucien Zhao <[email protected]>

- adapt for mcxe31x series - due to some bit defined in header files add some conditional macro to separate Signed-off-by: Lucien Zhao <[email protected]>

- add binding files: nxp,mcxe31x-siul2-pinctrl.yaml - Enable PINCTRL_NXP_SIUL2 when nxp,mcxe31x-siul2-pinctrl is ok Signed-off-by: Lucien Zhao <[email protected]>

- create 'mcxe' as family and 'mcxe31x' as series - add pinctrl_soc.h - add soc.c/.h to do some soc level initialization - add ecc initialization in mcxe31x_soc_initialization.S Signed-off-by: Lucien Zhao <[email protected]>

- Generate a full devices device tree file - Use specific_part.dtsi + full_devices.dtsi way to desribe all devices Signed-off-by: Lucien Zhao <[email protected]>

- support XIP way to boot - add board doc and picture - enable cases below: hello_world/blinky/button/ philosophers/synchronization/ gpio_basic_api Signed-off-by: Lucien Zhao <[email protected]>

dleach02 · 2025-10-08T21:28:00Z

@lucien-nxp I merged the HAL side PR and updated this PR for you.

sonarqubecloud · 2025-10-08T21:51:15Z

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

lucien-nxp force-pushed the add_mcxe31x_baseline_support branch from 6017e5b to 0d95a80 Compare September 5, 2025 15:53

lucien-nxp changed the title ~~Add mcxe31x baseline support~~ Add frdm_mcxe31 board basic support Sep 5, 2025

zephyrbot added platform: NXP Drivers NXP Semiconductors, drivers area: Clock Control area: Interrupt Controller platform: NXP NXP platform: NXP S32 NXP Semiconductors, S32 labels Sep 5, 2025

zephyrbot requested review from Dat-NguyenDuy, DerekSnell, EmilioCBen, JiafeiPan, PetervdPerk-NXP, ZhaoxiangJin, bperseghetti, butok, congnguyenhuu, dbaluta, decsny, dleach02, manuargue, mmahadevan108 and nordic-krch September 5, 2025 16:11

zephyrbot assigned nordic-krch Sep 5, 2025

github-actions bot added manifest manifest-hal_nxp DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Sep 5, 2025

lucien-nxp force-pushed the add_mcxe31x_baseline_support branch from 0d95a80 to 5d68882 Compare September 5, 2025 16:32

lucien-nxp changed the title ~~Add frdm_mcxe31 board basic support~~ Add frdm_mcxe31b board basic support Sep 5, 2025

lucien-nxp force-pushed the add_mcxe31x_baseline_support branch from 5d68882 to 3ef987b Compare September 5, 2025 18:10

lucien-nxp requested a review from ChayGuo September 18, 2025 09:50

nordicjm requested changes Sep 18, 2025

View reviewed changes

lucien-nxp force-pushed the add_mcxe31x_baseline_support branch 4 times, most recently from f2b7edb to d777dfa Compare September 20, 2025 13:47

hakehuang previously approved these changes Sep 22, 2025

View reviewed changes

lucien-nxp mentioned this pull request Sep 29, 2025

Add frdm mcxe247 basic support #96706

Open

lucien-nxp requested a review from nordicjm September 29, 2025 13:43

nordicjm reviewed Sep 30, 2025

View reviewed changes

nordicjm requested changes Sep 30, 2025

View reviewed changes

lucien-nxp added 7 commits October 8, 2025 16:25

west.yml: update hal_nxp for mcxe31x pinctrl.h

1a3bf7c

Updated hal_nxp to add mcxe31x pinctrl.h Signed-off-by: Lucien Zhao <[email protected]>

drivers: clock_control: add nxp_mc_cgm clock driver

c8937fb

- add clock_init function to initialize clock sources according devicetree settings - finish basic clock api function Signed-off-by: Lucien Zhao <[email protected]>

drivers: interrupt_controller: adapt for mcxe31x series

a14e5fe

- adapt for mcxe31x series - due to some bit defined in header files add some conditional macro to separate Signed-off-by: Lucien Zhao <[email protected]>

driver: pinctrl: adapt for mcxe31x series

f6429e3

- add binding files: nxp,mcxe31x-siul2-pinctrl.yaml - Enable PINCTRL_NXP_SIUL2 when nxp,mcxe31x-siul2-pinctrl is ok Signed-off-by: Lucien Zhao <[email protected]>

soc: nxp: mcx: add mcxe31 series soc

1e5ed70

- create 'mcxe' as family and 'mcxe31x' as series - add pinctrl_soc.h - add soc.c/.h to do some soc level initialization - add ecc initialization in mcxe31x_soc_initialization.S Signed-off-by: Lucien Zhao <[email protected]>

dts: arm: nxp: add mcxe31x device tree

e6963aa

- Generate a full devices device tree file - Use specific_part.dtsi + full_devices.dtsi way to desribe all devices Signed-off-by: Lucien Zhao <[email protected]>

boards： nxp: add frdm_mcxe31b board support

96f1e57

- support XIP way to boot - add board doc and picture - enable cases below: hello_world/blinky/button/ philosophers/synchronization/ gpio_basic_api Signed-off-by: Lucien Zhao <[email protected]>

dleach02 dismissed hakehuang’s stale review via 96f1e57 October 8, 2025 21:25

dleach02 force-pushed the add_mcxe31x_baseline_support branch from d777dfa to 96f1e57 Compare October 8, 2025 21:25

github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Oct 8, 2025

zephyrbot added platform: NXP MCU area: Boards/SoCs area: Tests Issues related to a particular existing or missing test area: Devicetree Bindings labels Oct 8, 2025

zephyrbot requested a review from nashif October 8, 2025 21:28

	DT_HAS_NXP_MCXE31X_SIUL2_PINCTRL_ENABLED
	DT_HAS_NXP_MCXE31X_SIUL2_PINCTRL_ENABLED

Add frdm_mcxe31b board basic support #95560

Are you sure you want to change the base?

Add frdm_mcxe31b board basic support #95560

Conversation

lucien-nxp commented Sep 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Sep 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hakehuang commented Sep 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

lucien-nxp commented Sep 29, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

dleach02 commented Oct 8, 2025

Uh oh!

sonarqubecloud bot commented Oct 8, 2025

Quality Gate passed

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

lucien-nxp commented Sep 5, 2025 •

edited

Loading

github-actions bot commented Sep 5, 2025 •

edited

Loading

hakehuang commented Sep 22, 2025 •

edited

Loading